-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: re-introduce a section from provider docs #2716
docs: re-introduce a section from provider docs #2716
Conversation
apps/docs-snippets/src/guide/provider/querying-the-chain.test.ts
Outdated
Show resolved
Hide resolved
apps/docs-snippets/src/guide/provider/querying-the-chain.test.ts
Outdated
Show resolved
Hide resolved
apps/docs-snippets/src/guide/provider/querying-the-chain.test.ts
Outdated
Show resolved
Hide resolved
apps/docs-snippets/src/guide/provider/querying-the-chain.test.ts
Outdated
Show resolved
Hide resolved
apps/docs-snippets/src/guide/provider/querying-the-chain.test.ts
Outdated
Show resolved
Hide resolved
apps/docs-snippets/src/guide/provider/querying-the-chain.test.ts
Outdated
Show resolved
Hide resolved
Hey @Torres-ssf! Can you check out? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems the preview is broken 🤔
This was an old deployment, it should be updated now :) |
I think it was not working because the preview deployment did not happen. I had to manually trigger it as it needs to be authorized by one of the official contributors. |
Locally, it is working. In my opinion, the issue is not due to the code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mvares We should have all sections heading using the same standard. The new ones are different from the existing ones:
Existent ones:
New ones:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work
getMessages
andgetMessageProof
documentation #2711Checklist
tests
to prove my changesdocs